HighContrastInverse: Fix suggested-action colors

Choose a suitable foreground color instead of hardcoding white.

https://bugzilla.gnome.org/show_bug.cgi?id=764170

diff --git a/gtk/theme/HighContrast/_common.scss b/gtk/theme/HighContrast/_common.scss
index e08ec7e869641abfdd3135355b4e26527b3bca8d..8a367b269996b56f6ff59517156a3a1d4b34eafd 100644 (file)
--- a/gtk/theme/HighContrast/_common.scss
+++ b/gtk/theme/HighContrast/_common.scss
@@ -555,10 +555,10 @@ button {
   }
 
   // Suggested and Destructive Action buttons
-  @each $b_type, $b_color in (suggested-action, $selected_bg_color),
-                             (destructive-action, $destructive_color) {
+  @each $b_type, $b_color, $f_color in (suggested-action, $selected_bg_color, $selected_fg_color),
+                             (destructive-action, $destructive_color, white) {
     &.#{$b_type} {
-      @include button(normal, $b_color, white);
+      @include button(normal, $b_color, $f_color);
 
       &.flat {
         @include button(undecorated);
@@ -566,23 +566,23 @@ button {
         color: $b_color; //FIXME: does it work on the dark variant?
       }
 
-      &:hover { @include button(hover, $b_color, white); }
+      &:hover { @include button(hover, $b_color, $f_color); }
 
       &:active,
-      &:checked { @include button(active, $b_color, white); }
+      &:checked { @include button(active, $b_color, $f_color); }
 
       &:backdrop,
       &.flat:backdrop {
-        @include button(backdrop, $b_color, white);
+        @include button(backdrop, $b_color, $f_color);
 
         &:active,
-        &:checked { @include button(backdrop-active, $b_color, white); }
+        &:checked { @include button(backdrop-active, $b_color, $f_color); }
 
         &:disabled {
           @include button(backdrop-insensitive);
 
           &:active,
-          &:checked { @include button(backdrop-insensitive-active, $b_color, white); }
+          &:checked { @include button(backdrop-insensitive-active, $b_color, $f_color); }
         }
       }
 
@@ -598,7 +598,7 @@ button {
         @include button(insensitive);
 
         &:active,
-        &:checked { @include button(insensitive-active, $b_color, white); }
+        &:checked { @include button(insensitive-active, $b_color, $f_color); }
       }
 
       .osd & {

diff --git a/gtk/theme/HighContrast/gtk-contained-inverse.css b/gtk/theme/HighContrast/gtk-contained-inverse.css
index 83dc0cceadafa7ef0a87064707fa9251bc9d30fe..4aef8db638401277759a57a4f2b58a593ea8bb7a 100644 (file)
--- a/gtk/theme/HighContrast/gtk-contained-inverse.css
+++ b/gtk/theme/HighContrast/gtk-contained-inverse.css
@@ -600,7 +600,7 @@ popover.background.touch-selection button, popover.background.magnifier button,
 button.suggested-action {
   border-width: 2px;
   border-style: solid;
-  color: white;
+  color: #000;
   background-image: none;
   background-color: #ddd;
   border-color: #9d9d9d; }
@@ -615,7 +615,7 @@ button.suggested-action {
   button.suggested-action:hover {
     border-width: 2px;
     border-style: solid;
-    color: white;
+    color: #000;
     background-color: #ddd;
     border-color: #9d9d9d;
     background-image: none; }
@@ -623,13 +623,13 @@ button.suggested-action {
     border-width: 2px;
     border-style: solid;
     background-image: none;
-    color: black;
+    color: white;
     background-color: #222222;
     border-color: #9d9d9d; }
   button.suggested-action:backdrop, button.suggested-action.flat:backdrop {
     border-width: 2px;
     border-style: solid;
-    color: white;
+    color: #000;
     background-color: #ddd;
     border-color: #ddd;
     background-image: none; }

diff --git a/gtk/theme/HighContrast/gtk-contained.css b/gtk/theme/HighContrast/gtk-contained.css
index 86af6bca517d6745bbad592eae17d01639a3d449..15c2b6c7acd69479dcbfc1486813d6c962c6170f 100644 (file)
--- a/gtk/theme/HighContrast/gtk-contained.css
+++ b/gtk/theme/HighContrast/gtk-contained.css
@@ -602,7 +602,7 @@ popover.background.touch-selection button, popover.background.magnifier button,
 button.suggested-action {
   border-width: 2px;
   border-style: solid;
-  color: white;
+  color: #fff;
   background-image: none;
   background-color: #000;
   border-color: black; }
@@ -617,7 +617,7 @@ button.suggested-action {
   button.suggested-action:hover {
     border-width: 2px;
     border-style: solid;
-    color: white;
+    color: #fff;
     background-color: #000;
     border-color: black;
     background-image: none; }
@@ -631,7 +631,7 @@ button.suggested-action {
   button.suggested-action:backdrop, button.suggested-action.flat:backdrop {
     border-width: 2px;
     border-style: solid;
-    color: white;
+    color: #fff;
     background-color: #000;
     border-color: #000;
     background-image: none; }